Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use GitHub Actions as the CI #187

Merged
merged 2 commits into from
Jan 11, 2024
Merged

Use GitHub Actions as the CI #187

merged 2 commits into from
Jan 11, 2024

Conversation

tombruijn
Copy link
Member

Move away from Semaphore and use GitHub Actions as our CI for this project.

Semaphore doesn't have Python 12 yet for us to test against, it will update versions once a month, and with GitHub Actions we can run it on the CI today.

[skip changeset]

Move away from Semaphore and use GitHub Actions as our CI for this
project.

Semaphore doesn't have Python 12 yet for us to test against, it will
update versions once a month, and with GitHub Actions we can run it on
the CI today.

[skip changeset]
@tombruijn
Copy link
Member Author

tombruijn commented Jan 11, 2024

This will need some more work, like:

  • auto cancel builds when you push a commit on a branch when another build for an older commit is already running. This can wait.
  • scheduled builds so we can see if anything fails
  • connect notifications to Slack if possible.

But this PR is here to gauge our interest.

@unflxw
Copy link
Contributor

unflxw commented Jan 11, 2024

I am certainly interested! I think this can replace our Semaphore usage as-is.

Before merging this we should actually see the checks fail as well as succeed. I don't trust it being green unless I have also seen it being red.

Run the build every Monday through Friday.
Copy link
Contributor

@unflxw unflxw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the other comment re: seeing the build fail.

After merging this we'll have to disable Semaphore for this repo so it doesn't mark each build as red.

@tombruijn
Copy link
Member Author

tombruijn commented Jan 11, 2024

@unflxw here's an example with a failure: https://github.com/appsignal/appsignal-python/actions/runs/7492027115

And yes to the Semaphore removal.

@tombruijn tombruijn merged commit b2a50a2 into main Jan 11, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants